Skip to content

Use proper MIT license#21

Open
johnnyshields wants to merge 1 commit intoisi-nlp:masterfrom
tablecheck:user-proper-mit-license
Open

Use proper MIT license#21
johnnyshields wants to merge 1 commit intoisi-nlp:masterfrom
tablecheck:user-proper-mit-license

Conversation

@johnnyshields
Copy link
Copy Markdown

@johnnyshields johnnyshields commented Mar 2, 2025

This project has a modified form of the MIT license which makes it a PITA to use at companies. Even though it would be basically fine to use, any project which does not have a FOSS license following the exact wording of the license needs a customized legal review (i.e. you have to pay lawyers.)

Specifically this language problematic:

Any publication of projects using uroman shall acknowledge its use: "This project uses the universal romanizer software 'uroman' written by Ulf Hermjakob, USC Information Sciences Institute (2015-2020)".
Bibliography: Ulf Hermjakob, Jonathan May, and Kevin Knight. 2018. Out-of-the-box universal romanization tool uroman. In Proceedings of the 56th Annual Meeting of Association for Computational Linguistics, Demo Track.

The clear intent of this language is to ensure the work (uroman) is cited in academic papers. It's better to move this requirement to the README instead.

Moving this wording to the README does not materially change the spirit of the license, nor does it impose any extra burden on users, so I think it's fine for the project maintainer to unilaterally accept it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant